Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

[CI] Upload the NAR package when pushing a tag #2035

Merged

Conversation

BewareMyPower
Copy link
Collaborator

No description provided.

Copy link

@BewareMyPower:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions github-actions bot added the doc-info-missing This pr needs to mark a document option in description label Dec 29, 2023
@BewareMyPower
Copy link
Collaborator Author

An example run: https://github.com/BewareMyPower/kop/actions/runs/7357165859

It only packages the NAR file, not including the oauth-clients and kafka-payload-processor packages, which should be used by importing a dependency to Maven.

Copy link

codecov bot commented Dec 29, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (aa2f171) 16.98% compared to head (003e6b3) 16.98%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2035   +/-   ##
=========================================
  Coverage     16.98%   16.98%           
  Complexity      730      730           
=========================================
  Files           191      191           
  Lines         14310    14310           
  Branches       1339     1339           
=========================================
  Hits           2430     2430           
  Misses        11704    11704           
  Partials        176      176           

@Demogorgon314 Demogorgon314 merged commit 6843a8e into streamnative:master Jan 4, 2024
22 checks passed
@BewareMyPower BewareMyPower deleted the bewaremypower/release-ci branch January 4, 2024 03:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
doc-info-missing This pr needs to mark a document option in description
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants